Skip to content

Conversation

@carojeandat
Copy link
Contributor

@carojeandat carojeandat commented Nov 4, 2025

PR Summary

Minor changes on the placeholder and the label of the search bar in modifications search dialog and nodes search dialog.

@carojeandat carojeandat requested a review from sBouzols November 4, 2025 16:17
Comment on lines 135 to 136
? 'rootNetwork.searchPlaceholder.nodes'
: 'rootNetwork.searchPlaceholder.modifications',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
? 'rootNetwork.searchPlaceholder.nodes'
: 'rootNetwork.searchPlaceholder.modifications',
? 'rootNetwork.searchLabel.nodes'
: 'rootNetwork.searchLabel.modifications',

Comment on lines 129 to 131
placeholder={intl.formatMessage({
id: 'searchPlaceholder',
})}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
placeholder={intl.formatMessage({
id: 'searchPlaceholder',
})}

remove placeholder ? the label usage is sufficient no ?

Copy link
Contributor

@sBouzols sBouzols left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review OK
Tests OK
Console warning check OK

@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 5, 2025

@carojeandat carojeandat merged commit c78c570 into main Nov 5, 2025
5 checks passed
@carojeandat carojeandat deleted the fix-grouped-modifications-search-results branch November 5, 2025 10:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants